Conversation
5 tasks
Collaborator
|
merged dependency PRs. let me know when to merge this one |
Collaborator
Author
|
ready to merge |
brenzi
pushed a commit
to integritee-network/worker
that referenced
this pull request
Jun 30, 2021
Poc of block broadcasting: Closes #244. It is only a PoC, as the workers simply broadcast blocks. There is no scheduling (aura) and received blocks by other workers are only logged to the command line without applying the state diff. Major: * Added `enclave-api` crate. PoC on how we can abstract the interface to the enclave. * Added `Worker` struct. PoC on how we could abstract every aspect of the worker's features. * Added PoC of a new `ws-server` the worker listens for direct requests. It comes from the [jsonrpsee](https://github.com/paritytech/jsonrpsee) crate - the rpc abstraction that will be successor to the current implementation in substrate. It currently coexists with the old direct_server, but I think we should replace the old one soon. It removes a lot of boilerplate code and is async. Since the introduction of the `async/await` syntax in rust, I believe we should go async, as it is much less verbose in general. * Added `Stf::apply_statediff()` it is unit tested but not yet integrated in the enclave. Minor: * better naming in the `direct_client` that actually reflects the functionality. * Skip-ra does register the enclave with the registry but without remote attestation report instead of skipping the process completely. This was made possible in integritee-network/integritee-node#20 * Fixed minor stuff in the local-setup. Local setup stably works for me with a node that skips remote attestation checks.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
skip-ias-check. See: feature-flag for remote-attestation check pallet-teerex#25Do not merge until this is merged: integritee-network/pallet-teerex#26
Then we update the dependency branch here.